Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ERL-467: nemos-images-reference-*: *: add checkbox-erlangen-classic snap #55

Merged
merged 7 commits into from
Dec 14, 2023

Conversation

IsaacJT
Copy link
Collaborator

@IsaacJT IsaacJT commented Oct 25, 2023

Add the custom Checkbox snap created for the project. This contains all of the project-specific test suites that we require.

@IsaacJT IsaacJT added the enhancement New feature or request label Oct 25, 2023
@IsaacJT IsaacJT added this to the NemOS Cycle 10 milestone Oct 25, 2023
@IsaacJT IsaacJT requested review from m-kat and ahresse October 25, 2023 13:24
@IsaacJT IsaacJT self-assigned this Oct 25, 2023
@IsaacJT
Copy link
Collaborator Author

IsaacJT commented Oct 26, 2023

Requires #54

@IsaacJT IsaacJT force-pushed the ERL-467-add-checkbox-erlangen branch from cb4da73 to 1ea8512 Compare November 9, 2023 08:55
@IsaacJT IsaacJT added the WIP Work In Progress label Nov 21, 2023
Add the custom Checkbox snap created for the project. This contains all
of the project-specific test suites that we require.

Signed-off-by: Isaac True <isaac.true@canonical.com>
@IsaacJT IsaacJT force-pushed the ERL-467-add-checkbox-erlangen branch from 1ea8512 to c7c915b Compare November 21, 2023 10:53
Signed-off-by: Isaac True <isaac.true@canonical.com>
@IsaacJT IsaacJT force-pushed the ERL-467-add-checkbox-erlangen branch from a05e2f8 to 5b40ccc Compare November 21, 2023 11:15
@IsaacJT IsaacJT removed the WIP Work In Progress label Nov 21, 2023
@IsaacJT
Copy link
Collaborator Author

IsaacJT commented Nov 21, 2023

Ready for review @ahresse @m-kat

Copy link
Collaborator

@ahresse ahresse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works for me on development profile for mantic reference image.

# snap list checkbox-erlangen-classic
Name                       Version  Rev  Tracking       Publisher            Notes
checkbox-erlangen-classic  0.1dev   2    latest/stable  ce-certification-qa  classic

Thank you Isaac for adding spread tests.

@IsaacJT IsaacJT force-pushed the ERL-467-add-checkbox-erlangen branch 5 times, most recently from ba9e839 to 53dfaee Compare November 23, 2023 08:04
If building a "reference" image, we want to use the development profile
in order to include all snaps.

Signed-off-by: Isaac True <isaac.true@canonical.com>
… installed

Signed-off-by: Isaac True <isaac.true@canonical.com>
Signed-off-by: Isaac True <isaac.true@canonical.com>
@IsaacJT IsaacJT force-pushed the ERL-467-add-checkbox-erlangen branch from 53dfaee to 211d75b Compare November 23, 2023 08:25
@IsaacJT
Copy link
Collaborator Author

IsaacJT commented Nov 23, 2023

CI fixed

Copy link

@zongminl zongminl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some changes required, please see my inline comments

nemos-images-reference-lunar/qemu-amd64/config.sh Outdated Show resolved Hide resolved
nemos-images-reference-lunar/qemu-amd64/config.sh Outdated Show resolved Hide resolved
nemos-images-reference-lunar/qemu-arm64/config.sh Outdated Show resolved Hide resolved
nemos-images-reference-lunar/qemu-arm64/config.sh Outdated Show resolved Hide resolved
nemos-images-reference-lunar/s32g274ardb2/config.sh Outdated Show resolved Hide resolved
nemos-images-reference-mantic/qemu-amd64/config.sh Outdated Show resolved Hide resolved
nemos-images-reference-mantic/qemu-arm64/config.sh Outdated Show resolved Hide resolved
nemos-images-reference-mantic/qemu-arm64/config.sh Outdated Show resolved Hide resolved
nemos-images-reference-mantic/s32g274ardb2/config.sh Outdated Show resolved Hide resolved
nemos-images-reference-mantic/s32g274ardb2/config.sh Outdated Show resolved Hide resolved
Signed-off-by: Alexandre Esse <alexandre.esse@canonical.com>
…pendency

`checkbox` snap has launchers built-in but the one we use are in
`checkbox-erlangen-classic` which has launchers and providers (in checkbox
term, the jobs and testplans), but doesn't have checkbox-runtime so we still
depend on `checkbox22` to provide the service and also the generic providers.

Signed-off-by: Alexandre Esse <alexandre.esse@canonical.com>
@IsaacJT
Copy link
Collaborator Author

IsaacJT commented Nov 30, 2023

LGTM, thanks for the additions

Copy link

@zongminl zongminl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1, thank you

@ahresse ahresse merged commit b1052ee into master Dec 14, 2023
13 checks passed
@ahresse ahresse deleted the ERL-467-add-checkbox-erlangen branch December 14, 2023 17:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants